Skip to content

feat: add aws-amplify plugin#18

Open
must1d wants to merge 1 commit intoawslabs:mainfrom
must1d:aws-amplify
Open

feat: add aws-amplify plugin#18
must1d wants to merge 1 commit intoawslabs:mainfrom
must1d:aws-amplify

Conversation

@must1d
Copy link

@must1d must1d commented Feb 10, 2026

Related

N/A

Changes

Add the aws-amplify plugin for building full-stack apps with AWS Amplify Gen 2.

  • Marketplace: Register aws-amplify plugin in .claude-plugin/marketplace.json
  • Plugin manifest: plugins/aws-amplify/.claude-plugin/plugin.json with metadata (author, license, keywords)
  • MCP server: .mcp.json configuring aws-mcp (via mcp-proxy-for-aws) for SOP retrieval
  • Slash command: commands/build.md/aws-amplify:build for explicit invocation
  • Auto-triggered skill: skills/amplify-workflow/SKILL.md — orchestrates a phased workflow (Backend → Sandbox →
    Frontend → Testing → Production) driven by official AWS SOPs
  • README: Documentation covering installation, usage, prerequisites, and plugin structure

Acknowledgment

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.

hfurkanbozkurt
hfurkanbozkurt previously approved these changes Feb 10, 2026
@must1d must1d changed the title add aws-amplify plugin feat: add aws-amplify plugin Feb 10, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new aws-amplify plugin to the repository’s plugin marketplace, intended to guide developers through AWS Amplify Gen 2 full-stack workflows using SOP-driven phases and an MCP-backed SOP retrieval server.

Changes:

  • Registers the aws-amplify plugin in the central marketplace registry.
  • Adds plugin assets: manifest, MCP server config, slash command, and an auto-invoked “amplify-workflow” skill.
  • Adds plugin-level documentation describing installation and usage.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
plugins/aws-amplify/skills/amplify-workflow/SKILL.md Adds the phased Amplify Gen 2 workflow skill that orchestrates SOP-driven execution.
plugins/aws-amplify/commands/build.md Adds /aws-amplify:build slash command delegating to the workflow skill.
plugins/aws-amplify/README.md Adds end-user documentation for installing and using the plugin.
plugins/aws-amplify/.mcp.json Adds MCP server configuration intended for AWS SOP retrieval.
plugins/aws-amplify/.claude-plugin/plugin.json Adds the plugin manifest (metadata for marketplace/tooling).
.claude-plugin/marketplace.json Registers aws-amplify in the marketplace registry.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings February 11, 2026 09:31
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings February 11, 2026 09:53
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@must1d must1d requested review from a team as code owners February 11, 2026 11:09
Copilot AI review requested due to automatic review settings February 11, 2026 11:12
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings February 11, 2026 11:26
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 9 out of 9 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

Copilot AI review requested due to automatic review settings March 11, 2026 18:54
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

@scoropeza
Copy link
Contributor

Code review

Found 1 issue:

  1. Unresolved merge conflict marker (=======) in README.md between the aws-amplify MCP Servers section and the migration-to-aws section. This will break the rendered documentation and indicates the branch has conflicts that need to be resolved before merging. The PR also shows mergeable: CONFLICTING status.

agent-plugins/README.md

Lines 144 to 148 in ee2e90a

| ----------- | -------------------------------------- |
| **aws-mcp** | AWS documentation and service guidance |
=======
## migration-to-aws

🤖 Generated with Claude Code

- If this code review was useful, please react with 👍. Otherwise, react with 👎.

@scoropeza
Copy link
Contributor

Code review

Found 4 issues:

  1. Unresolved merge conflict marker (=======) in README.md between the aws-amplify MCP Servers section and the migration-to-aws section. This will break rendered documentation.

agent-plugins/README.md

Lines 144 to 148 in ee2e90a

| ----------- | -------------------------------------- |
| **aws-mcp** | AWS documentation and service guidance |
=======
## migration-to-aws

  1. SKILL.md tells users to "come back and start a new conversation" after configuring AWS credentials (line 45). This is unnecessarily disruptive and not enforceable -- the agent can re-run the prerequisite check in the same session. Consider changing to: "Re-run the prerequisite check script to verify your credentials, then continue."

**Quick options:**
- Run `aws configure` to set up access keys
- Run `aws sso login` if using AWS IAM Identity Center
Once your credentials are configured, **come back and start a new conversation** to continue building with Amplify.
```

  1. SKILL.md is missing an explicit ## Defaults section. All other plugins include one (DESIGN_GUIDELINES says "All defaults explicitly specified"). The amplify workflow has implicit defaults (phase ordering, framework choices, deployment targets) that should be documented with override syntax. See deploy-on-aws SKILL.md for reference.

https://github.com/awslabs/agent-plugins/blob/ee2e90a81dd83769842c9b49c83dcdd638af95f1/plugins/aws-amplify/skills/amplify-workflow/SKILL.md

  1. prereq-check.sh uses aws sts get-caller-identity (line 35) but AWS CLI is not listed as a prerequisite. The script checks Node.js, npm, and AWS credentials -- but if aws is not installed, the credentials check fails with a confusing error rather than a clear "AWS CLI not found" message. Consider adding an explicit AWS CLI check to the script and documenting it as a prerequisite.

# AWS credentials configured
check "AWS credentials" bash -c 'AWS_PAGER="" aws sts get-caller-identity --query Account --output text'

🤖 Generated with Claude Code

- If this code review was useful, please react with 👍. Otherwise, react with 👎.

@hfurkanbozkurt
Copy link

Thanks for checking @scoropeza , I think we will get this to finish line together. Could you please take another look and let me know?

Copilot AI review requested due to automatic review settings March 12, 2026 21:35
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

@scottschreckengaust scottschreckengaust requested review from a team, scoropeza and theagenticguy and removed request for krokoko March 12, 2026 21:38
@hfurkanbozkurt hfurkanbozkurt self-requested a review March 12, 2026 21:41
hfurkanbozkurt
hfurkanbozkurt previously approved these changes Mar 12, 2026
@scoropeza
Copy link
Contributor

Code review

All 4 issues from the previous review have been addressed:

  1. Merge conflict resolved (PR is now mergeable)
  2. "Start new conversation" replaced with re-run prerequisite check flow
  3. ## Defaults section added to SKILL.md
  4. AWS CLI check added to prereq-check.sh

One remaining item: the plugin is missing a plugins/aws-amplify/README.md. DESIGN_GUIDELINES requires this file with the following sections:

  • Overview - 1-2 sentence plugin purpose
  • Skills - List of included skills with descriptions
  • MCP Servers - Required servers and what they provide
  • Installation - claude plugin install command
  • Examples - 3-5 concrete usage examples

Keep under 300 lines. See plugins/migration-to-aws/README.md for reference.

🤖 Generated with Claude Code

- If this code review was useful, please react with 👍. Otherwise, react with 👎.

Copilot AI review requested due to automatic review settings March 13, 2026 22:24
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

@hfurkanbozkurt hfurkanbozkurt force-pushed the aws-amplify branch 2 times, most recently from 9a95bad to e317434 Compare March 13, 2026 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants